superbench/benchmarks/micro_benchmarks/gpu_stream/gpu_stream_utils.cpp:90

• ParseOpts initializes size_specified to true, which means omitting --size will not trigger the missing-required-arg check, unlike the pattern used in other micro-benchmarks (e.g., cpu_copy/gpu_copy). If --size is intended to be required per PrintUsage, size_specified should start as false (and only be set true when parsed).

                                     {"check_data", no_argument, nullptr, static_cast<int>(OptIdx::kEnableCheckData)},
                                     {"data_type", required_argument, nullptr, static_cast<int>(OptIdx::kDataType)}};
    int getopt_ret = 0;
    int opt_idx = 0;
    bool size_specified = true;
    bool num_warm_up_specified = false;
    bool num_loops_specified = false;

superbench/benchmarks/micro_benchmarks/gpu_stream/gpu_stream_utils.cpp:99

• size_specified is initialized to true, which makes the "required option" validation (if (!size_specified || ...)) ineffective for --size and inconsistent with PrintUsage() (which shows --size as required). Either initialize size_specified to false (to enforce --size) or remove size_specified from the required-check and update the usage text accordingly.

    int getopt_ret = 0;
    int opt_idx = 0;
    bool size_specified = true;
    bool num_warm_up_specified = false;
    bool num_loops_specified = false;

    bool parse_err = false;
    while (true) {
        getopt_ret = getopt_long(argc, argv, "", options, &opt_idx);
        if (getopt_ret == -1) {
            if (!size_specified || !num_warm_up_specified || !num_loops_specified) {
                parse_err = true;
            }
            break;

superbench/benchmarks/micro_benchmarks/gpu_stream/gpu_stream_utils.cpp:89

• size_specified is initialized to true, so the required-argument validation (if (!size_specified || ...)) can never fail due to a missing --size. This is inconsistent with other micro-benchmark option parsers in the repo (which start this flag as false) and with PrintUsage() indicating --size is required. Initialize size_specified to false (or remove the flag entirely if --size is intended to be optional) so missing/invalid size handling is unambiguous.

                                     {"data_type", required_argument, nullptr, static_cast<int>(OptIdx::kDataType)}};
    int getopt_ret = 0;
    int opt_idx = 0;
    bool size_specified = true;
    bool num_warm_up_specified = false;

superbench/benchmarks/micro_benchmarks/gpu_stream/gpu_stream_utils.cpp:92

• ParseOpts initializes size_specified to true, which makes --size effectively optional even though the usage string lists it as required and other micro-benchmarks start this flag as false. This is likely unintended and makes the required-argument validation inconsistent. Initialize size_specified to false (or update the required-argument logic/usage text if --size is intentionally optional due to the default).

    int getopt_ret = 0;
    int opt_idx = 0;
    bool size_specified = true;
    bool num_warm_up_specified = false;
    bool num_loops_specified = false;

    bool parse_err = false;